ScreenShield: make .lock() and .unlock() idempotent

Those methods can be called multiple times in certain situations,
and will be even more so after hooking them to the system bus.

diff --git a/js/ui/screenShield.js b/js/ui/screenShield.js
index f375417692593f768b7fe4fe44da77fafcf96ac5..c4a23b6e9070e2a0dabebe5ad6b19ecaaf4a0782 100644 (file)
--- a/js/ui/screenShield.js
+++ b/js/ui/screenShield.js
@@ -616,11 +616,13 @@ const ScreenShield = new Lang.Class({
 
         this._lightbox.hide();
 
-        Main.popModal(this.actor);
-        this.actor.hide();
+        if (this._isModal) {
+            Main.popModal(this.actor);
+            this._isModal = false;
+        }
 
-        this._isModal = false;
         this._isLocked = false;
+        this.actor.hide();
 
         this.emit('lock-status-changed', false);
     },